Skip to content

Conversation

res0nance
Copy link

@res0nance res0nance commented Sep 19, 2025

Hey @nfalco79 I'm using this PR as a springboard to start a discussion on debuggability. I've recently had issues with understanding why builds are not triggered and currently its impossible to figure out if a webhook is being processed with what payload and is a build even triggered and if not why isn't a build triggered.

It seems like restarting jenkins in some cases fixes the issue of builds being triggered but perhaps its better for us to get an understanding as to why this happens and we can avoid having such restarts.

Your checklist for this pull request

  • Make sure you are requesting to pull a topic/feature/bugfix branch (right side) and not your master branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or in Jenkins JIRA
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Did you provide a test-case? That demonstrates feature works or fixes the issue.

@nfalco79
Copy link
Member

nfalco79 commented Sep 19, 2025

There is already an issue in JIRA about this and a PR that add logs to understand where the webhook processing and/or generated event could be lost.
PR #1105

@res0nance
Copy link
Author

There is already an issue in JIRA about this and a PR that add logs to understand where the webhook processing and/or generated event could be lost. PR #1105

Should those logs be added in a release? Would likely allow for more people to easily pick up such changes and assist with the debugging effort.

@nfalco79
Copy link
Member

No there are only to debug a specific issue

@res0nance
Copy link
Author

No there are only to debug a specific issue

Without such logs users would have a hard time being able to have any visibility into the webhook processing. I've commented in #1105 on the difficulty of trying to diagnose any such issue. IMO it would be helpful to keep such logs perhaps on the FINE level so it doesn't appear by default

@nfalco79
Copy link
Member

PR #1121 should cover any case about webhook processor phase

@nfalco79 nfalco79 closed this Sep 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants